-
-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[stable27] ocm services #40586
[stable27] ocm services #40586
Conversation
5ae9081
to
19f7fa8
Compare
@provokateurin do you think #40575 will fix psalm analysis ? should we cherrypick ? |
No my PR won't fix it. You need to remove all the new |
ef4d72e
to
1af7102
Compare
I think you should remove the annotation instead of supressing it |
Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
1af7102
to
0fac750
Compare
For info, NC 27.1.2 was deployed at PSNC in Poland, a site of the ScienceMesh (the infra where the initial report #39001 originated), and I can confirm basic sharing functionally works as expected. Thanks for the effort! |
backport of #39574